Skip to content

[OGUI-1913] Reset test profile DB before each test run to fix alternating pass/fail - #3563

Merged
isaachilly merged 2 commits into
devfrom
improv/ILG/OGUI-1913/Fix-local-user-profile-test-reliability
Aug 4, 2026
Merged

[OGUI-1913] Reset test profile DB before each test run to fix alternating pass/fail#3563
isaachilly merged 2 commits into
devfrom
improv/ILG/OGUI-1913/Fix-local-user-profile-test-reliability

Conversation

@isaachilly

@isaachilly isaachilly commented Aug 3, 2026

Copy link
Copy Markdown
Collaborator

testdb.json is used by the test server to persist user profiles, but it is never cleaned up between separate test runs (npm run test).

successfully save the profile of the user when pressed the "Save Profile" menu-item test in user-actions-mocha.js toggles a column's visibility and saves it. Because the file isn't reset between runs, this modified state carries over to the next time the suite runs, causing the subsequent test, which relies on the visibility being what it originally was, to fail on that next run.

I have JIRA issue created

  • branch and/or PR name(s) includes JIRA ID
  • issue has "Fix version" assigned
  • issue "Status" is set to "In review"
  • PR labels are selected
  • FLP integration tests were ran successful

`testdb.json` is used by the test server to persist user profiles, but it is never cleaned up between separate test runs (`npm run test`).

`successfully save the profile of the user when pressed the "Save Profile" menu-item` test in `user-actions-mocha.js` toggles a column's visibility and saves it. Because the file isn't reset between runs, this modified state carries over to the next time the suite runs, causing the subsequent test, which relies on the visibility being what it originally was, to fail on that next run.
@isaachilly isaachilly self-assigned this Aug 3, 2026
@isaachilly
isaachilly marked this pull request as ready for review August 3, 2026 16:38
@isaachilly
isaachilly requested a review from graduta as a code owner August 3, 2026 16:38

@graduta graduta left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. The only remark I have to make sure that in the commit you mention this is needed for local testing.
In GH Actions this is not an issue as we always start from a clean state.

@isaachilly
isaachilly merged commit 5ea3dab into dev Aug 4, 2026
9 checks passed
@isaachilly
isaachilly deleted the improv/ILG/OGUI-1913/Fix-local-user-profile-test-reliability branch August 4, 2026 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants